Skip to content
This repository was archived by the owner on Dec 6, 2024. It is now read-only.

Removes bucketName fields from Protocol #41

Merged
merged 2 commits into from
May 7, 2021

Conversation

krishchow
Copy link

@krishchow krishchow commented May 4, 2021

Fixes: #38
Fixes: #36

This PR aims to remove the bucketName fields in the protocol in favor of the bucketID field in the Bucket.spec. We also remove the API-approved annotation.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 4, 2021
@k8s-ci-robot k8s-ci-robot requested review from rrati and saad-ali May 4, 2021 20:32
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 4, 2021
@krishchow
Copy link
Author

/assign wlan0

@wlan0
Copy link
Contributor

wlan0 commented May 7, 2021

/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: krishchow, wlan0

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm "Looks good to me", indicates that a PR is ready to be merged. labels May 7, 2021
@k8s-ci-robot k8s-ci-robot merged commit fac7c50 into kubernetes-retired:master May 7, 2021
NicolasT added a commit to scality/cosi-driver-sample that referenced this pull request May 10, 2021
Recent changes in the API removed the `endpoint` and `bucketName` fields
from the `S3Protocol` structure (in the CRD). This change reflects the
API change so CI succeeds again.

See: kubernetes-retired/container-object-storage-interface-api#41
BlaineEXE pushed a commit to BlaineEXE/cosi-api that referenced this pull request May 29, 2024
shanduur pushed a commit to shanduur/container-object-storage-interface-api that referenced this pull request Jun 6, 2024
shanduur pushed a commit to shanduur/container-object-storage-interface-api that referenced this pull request Jun 6, 2024
BlaineEXE pushed a commit to BlaineEXE/cosi-api that referenced this pull request Jun 14, 2024
BlaineEXE pushed a commit to BlaineEXE/cosi-api that referenced this pull request Jun 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
3 participants